-
-
Notifications
You must be signed in to change notification settings - Fork 47.1k
Data Transformations #2191
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Data Transformations #2191
Conversation
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
Hey @danmurphy1217, TravisCI finished with status TravisBuddy Request Identifier: b937ffc0-c218-11ea-8e31-7fd6e6dac888 |
Looks like my old commits are getting grouped into this pull request- sorry about that. Should I close this and open a new one for just the data_transformations.py file? |
In the markdown checklist above, adding spaces on either side of the
On the code, could we get the mean and standard deviation from https://docs.python.org/3/library/statistics.html and not round them? Rounding multiple times an a single calculation worries me. |
Yes, I can change that. I am going to close this pull request and open a new one with the current version of the Python Algorithms repository (to avoid conflicts with my previous commits). That ok? |
Sure. If you cannot rebase then close and open a new PR. Just add Replaces #2191 so we can follow the thread of comments. |
Describe your change:
Checklist:
Fixes: #{$ISSUE_NO}
.